Skip to content

Fix CI deprecation warning#959

Merged
edenhaus merged 1 commit into
devfrom
llvm-deprecate-warning
May 9, 2025
Merged

Fix CI deprecation warning#959
edenhaus merged 1 commit into
devfrom
llvm-deprecate-warning

Conversation

@edenhaus

@edenhaus edenhaus commented May 9, 2025

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 9, 2025 14:35
@edenhaus edenhaus added the pr: refactor PR with code refactoring label May 9, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the CI configuration to resolve a deprecation warning by modifying the cargo llvm-cov command.

  • Replaces the deprecated "--no-run" option with the "report" subcommand.
  • Updates the command arguments to ensure proper coverage reporting.

@codecov

codecov Bot commented May 9, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.31%. Comparing base (c34b9f2) to head (17bda4e).
Report is 2 commits behind head on dev.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #959   +/-   ##
=======================================
  Coverage   93.31%   93.31%           
=======================================
  Files         128      128           
  Lines        4993     4993           
  Branches      330      330           
=======================================
  Hits         4659     4659           
  Misses        274      274           
  Partials       60       60           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq

codspeed-hq Bot commented May 9, 2025

Copy link
Copy Markdown

CodSpeed Performance Report

Merging #959 will not alter performance

Comparing llvm-deprecate-warning (17bda4e) with dev (2b18d80)

Summary

✅ 6 untouched benchmarks

@edenhaus edenhaus merged commit 60fb4d1 into dev May 9, 2025
27 checks passed
@edenhaus edenhaus deleted the llvm-deprecate-warning branch May 9, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: refactor PR with code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants